Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Run tests on .NET 8.0 #2039

Merged
merged 7 commits into from
Nov 15, 2023
Merged

Run tests on .NET 8.0 #2039

merged 7 commits into from
Nov 15, 2023

Conversation

idg10
Copy link
Collaborator

@idg10 idg10 commented Nov 14, 2023

Added net8.0 and net8.0-windows10.0.19041 targets to test project.

Modified Directory.build.targets to recognize .NET 8.0 targets.

One test needed to be lightly modified because the way in which it was trying to test what it wanted to test doesn't quite work in .NET 8.0.

One test needed to be lightly modified because the way in which it was trying to test what it wanted to test doesn't quite work in .NET 8.0.
Copy link
Collaborator

@mwadams mwadams left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This removes test support for dotnet7.0?

@idg10 idg10 merged commit bd4fb9e into main Nov 15, 2023
7 checks passed
@idg10 idg10 deleted the feature/net80-tests branch November 15, 2023 08:04
@idg10
Copy link
Collaborator Author

idg10 commented Nov 22, 2023

@mwadams wrote:

This removes test support for dotnet7.0?

It's certainly not supposed to! I had a quick look through the PR to double check, and as far as I can tell this just adds .NET 8.0 in the test projects.

I was planning to remove .NET 7.0 from the tests once .NET 7.0 goes out of support, but not before.

idg10 added a commit that referenced this pull request May 21, 2024
* Run tests on .NET 8.0
* Add .NET 8.0 installation to build pipeline
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants